-
Notifications
You must be signed in to change notification settings - Fork 224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
initial commit for free agent auction report #59
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #59 +/- ##
===========================================
- Coverage 91.37% 79.31% -12.07%
===========================================
Files 12 13 +1
Lines 545 638 +93
===========================================
+ Hits 498 506 +8
- Misses 47 132 +85
Continue to review full report at Codecov.
|
Thanks for posting a PR! I will try to look at this fully later today. One note, could you make a unit test for this new code. Also I have never played in a FAAB league before but do you know if you can get this data through the recent_activity endpoint? |
Very interesting! I will also try out that endpoint in my non FAAB league and see what else it returns. |
I've actually been looking into including offers report into this module, and apparently everything I need has been here already! I'm curious to know what needs to be worked on here to get this integrated/merged? |
This PR was created before the package was refactored for multiple sports. This endpoint can be used for all league types to get wavier report. I think we could do something similar to how baseball created different box_score categories depending on the league settings. So we could do the same for wavier report. Create a wavier report class and have it add different data depending on if the League is FABB or just normal wavier claims. |
I wanted a function that would allow me to automatically post the Free Agent Auction report to Slack/GroupMe/etc/. I'm only in leagues that use FAAB. This could be generalized to include other waiver systems, but I would need an example public league to test on.
This is my first PR, so let me know if anything looks off!